-
Notifications
You must be signed in to change notification settings - Fork 1.4k
manifest: update collab-nrfx-4.0 branch to have error codes aligned #25155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: collab-nrfx-4.0
Are you sure you want to change the base?
manifest: update collab-nrfx-4.0 branch to have error codes aligned #25155
Conversation
|
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 3 projects with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
CI InformationTo view the history of this post, click the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 81d6c66a73324e56611d53d5de519721da5090b1 more detailssdk-nrf:
nrfxlib:
zephyr:
Github labels
List of changed files detected by CI (24)Outputs:ToolchainVersion: 2b2cd9579a Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
08c1a7f to
389abb8
Compare
e6f01f8 to
507ffc9
Compare
| groups: | ||
| - nrfx | ||
| revision: 438fc5f49a2cfdef455b3f3b273814489ff7a571 | ||
| revision: pull/987/head |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@masz-nordic is there a point in keeping and updating this SHA, if #25359 was merged to collab? It won't be used in that case, unless I am missing something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To test properly you need to revert this commit:
5ce1b7f
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Memory footprint analysis revealed the following potential issuesapplications.nrf_desktop.zdebug[nrf52833dk/nrf52833]: ROM size increased by 576[B] in comparison to the main[1cee1cc] branch. - link (cc: @nrfconnect/ncs-si-bluebagel) Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-25155/16) |
Manifest update. Signed-off-by: Michał Stasiak <[email protected]>
NRFX NVMC driver now returns errno values. Signed-off-by: Michał Stasiak <[email protected]>
NRFX RTC driver now returns errno values. Signed-off-by: Michał Stasiak <[email protected]>
nrfx_err_t is to be removed from nrfx completely. Signed-off-by: Michał Stasiak <[email protected]>
This reverts commit 5ce1b7f.
a2e32d8 to
81d6c66
Compare
Manifest update.
test-sdk-nrf: collab-nrfx-4.0